Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate package manager from Yarn to PNPM #272

Merged
merged 1 commit into from
Sep 14, 2024

Conversation

elchininet
Copy link
Collaborator

@elchininet elchininet commented Sep 14, 2024

We have been using Yarn as a package manager for some time, but since a while, PNPM has demonstrated to be a better alternative in terms of speed, disk space efficiency, dependency isolation and security.

This pull request migrates the package manager of the repo from Yarn to PNPM to get the advantages that it brings.

Note: as part of this pull request, the id of the Docker Cache was using the old .playwright_docker_version file that doesn't exist anymore in the repository. Now the version of the Playwright Docker image is taken also from the package.json using a bash script.

@elchininet elchininet force-pushed the migrate_package_manager_to_pnpm branch 5 times, most recently from ed30afe to 3e8ebb0 Compare September 14, 2024 15:01
@elchininet elchininet force-pushed the migrate_package_manager_to_pnpm branch from 3e8ebb0 to 212af54 Compare September 14, 2024 15:03
@elchininet elchininet marked this pull request as ready for review September 14, 2024 15:08
@NemesisRE NemesisRE merged commit 65fb255 into master Sep 14, 2024
5 checks passed
@NemesisRE NemesisRE deleted the migrate_package_manager_to_pnpm branch September 14, 2024 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants